From: Marius Hoch Date: Sun, 3 Jan 2016 19:31:05 +0000 (+0100) Subject: Slightly simplify QueryPage::outputResults X-Git-Tag: 1.31.0-rc.0~8449^2 X-Git-Url: http://git.cyclocoop.org/%22.%24info%5B?a=commitdiff_plain;h=08548cbf39d8df4c601f6839284e1588c3b76131;p=lhc%2Fweb%2Fwiklou.git Slightly simplify QueryPage::outputResults By removing/ deprecating unused things. Change-Id: Ib9f7439eeeb261a60903eb23f13c65038ebe4945 --- diff --git a/includes/specialpage/QueryPage.php b/includes/specialpage/QueryPage.php index 2d25710614..27e645afcb 100644 --- a/includes/specialpage/QueryPage.php +++ b/includes/specialpage/QueryPage.php @@ -275,11 +275,14 @@ abstract class QueryPage extends SpecialPage { } /** - * Some special pages (for example SpecialListusers) might not return the + * Some special pages (for example SpecialListusers used to) might not return the * current object formatted, but return the previous one instead. * Setting this to return true will ensure formatResult() is called * one more time to make sure that the very last result is formatted * as well. + * + * @deprecated since 1.27 + * * @return bool */ function tryLastResult() { @@ -660,12 +663,9 @@ abstract class QueryPage extends SpecialPage { // @codingStandardsIgnoreEnd $line = $this->formatResult( $skin, $row ); if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; $html[] = $this->listoutput ? $line - : "{$line}\n"; + : "
  • {$line}
  • \n"; } } @@ -674,12 +674,9 @@ abstract class QueryPage extends SpecialPage { $row = null; $line = $this->formatResult( $skin, $row ); if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; $html[] = $this->listoutput ? $line - : "{$line}\n"; + : "
  • {$line}
  • \n"; } }